-
Notifications
You must be signed in to change notification settings - Fork 61
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Update air conditioning example #801
base: master
Are you sure you want to change the base?
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #801 +/- ##
=======================================
Coverage 93.54% 93.54%
=======================================
Files 26 26
Lines 3519 3519
=======================================
Hits 3292 3292
Misses 227 227 ☔ View full report in Codecov by Sentry. |
@odow how do you do format checks locally? |
SDDP.jl/.github/workflows/format_check.yml Lines 23 to 27 in b4c0a25
|
But I assumed @bernardokp wanted the examples fleshed out into proper literate tutorials (like the JuMP ones), not just some extra prints added. |
Yeah, that would be ideal. |
I mean, that would be a luxury!!! |
That will take some time. I taught it was just an small change, I don't have this time to spare right now. |
That's why I haven't done it 😄 |
If the goal is to create a tutorial, I can't promise anything now. @bernardokp you can help create the text? At least give me an initial version, I will extend it and create the code for it. |
I certainly can help. As a matter of fact, it would be nice if we could create sort of a procedure to do it because I will probably create more examples in the very near future. |
Is that what you want, @Thuener ? The air conditioning example first appeared in the book Introduction to Stochastic Programming, by Birge and Louveaux, 1997, Springer-Verlag, New York. It is on page 237, example 1. For a rescaled problem, they report an optimal first-stage solution of |
Closes #798